Description

This PR addresses Issue #9048 regarding slow codebase indexing by explicitly excluding .git directories from file listing operations.

Problem

Users reported slow codebase indexing, specifically mentioning that large .git folders were being processed. While .git was already in the DIRS_TO_IGNORE list, there were scenarios where it could still be included:

• When explicitly targeting hidden directories or directories in the ignore list, the code would use --no-ignore-vcs and --no-ignore flags
• These flags would override the normal .gitignore behavior, potentially allowing .git contents to slip through

Solution

• Added explicit .git exclusion as a critical priority pattern that is ALWAYS applied, regardless of other settings
• This ensures .git is excluded even when --no-ignore-vcs and --no-ignore flags are used for other purposes
• Applied the exclusion in both recursive and non-recursive modes

Changes

1. src/services/glob/list-files.ts

   • Added explicit -g '!**/.git/**' pattern in recursive mode
   • Added explicit -g '!.git' and -g '!.git/**' patterns in non-recursive mode
   • These patterns are applied before other exclusions to ensure priority

2. src/services/glob/tests/list-files-git-exclusion.spec.ts

   • Added comprehensive tests covering:
     • .git exclusion in recursive mode
     • .git exclusion in non-recursive mode
     • .git exclusion when targeting hidden directories
     • .git exclusion when .git itself is the target directory

Impact

• Prevents indexing of .git folder contents which can be very large in repositories with extensive history
• Improves codebase indexing performance
• No breaking changes - this is a performance optimization

Testing

• All existing tests pass ✅
• New tests added specifically for .git exclusion ✅
• Tests verify that .git is excluded in various scenarios

Notes

While .git was already in DIRS_TO_IGNORE, this explicit exclusion provides an extra layer of protection and ensures consistent behavior across all code paths. The user also mentioned wanting to exclude "these folders and files" (plural), suggesting they may want a more general exclusion mechanism. If needed, we can explore adding a configuration option for custom exclusions in a follow-up PR.

Fixes #9048